Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support pull_request_target event type #63

Merged
merged 1 commit into from
May 29, 2024

Conversation

Fishbowler
Copy link

The pull_request_target event is an alternative to the pull_request event but with some different security considerations. Docs.

We use it on private repositories to allow dependabot access to our secrets, safe in the knowledge that a private repo can't expose our secrets to drive-by malicious PRs.

This is a simple change that extends the logic for "pull_request" to "pull_request_target" for comparisons of changes, etc.

@romeovs romeovs merged commit 3a33d15 into romeovs:master May 29, 2024
pcaversaccio added a commit to pcaversaccio/createx that referenced this pull request Dec 18, 2024
### 🕓 Changelog

This PR updates the [`LCOV`](https://github.com/linux-test-project/lcov)
version to the latest release,
[`2.2`](https://github.com/linux-test-project/lcov/releases/tag/v2.2),
in the `coverage` CI pipeline. It also introduces a temporary workaround
for the `Verify minimum coverage` step, as `LCOV` `2.2` is not yet
supported in
[`zgosalvez/github-actions-report-lcov@v4`](https://github.com/zgosalvez/github-actions-report-lcov/tree/v4),
as outlined in issue
[#168](zgosalvez/github-actions-report-lcov#168).
Additionally, the
[`romeovs/[email protected]`](https://github.com/romeovs/lcov-reporter-action)
has been updated and is now triggered for `pull_request_target` events
as well (see PR
[#63](romeovs/lcov-reporter-action#63)).

---------

Signed-off-by: Pascal Marco Caversaccio <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants